Implement default reports#62
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Implements the Angular v21 “Default Reports” Insights page, porting the prior AngularJS inventory reports workflow into a standalone Angular component with Chart.js visualizations, report configuration management, and data export.
Changes:
- Added Default Reports page with scatter + aggregated bar charts, reactive controls, statistics display, and export/chart download actions.
- Added modals for report configuration CRUD and XLSX export naming/download.
- Added new API services/types for inventory reports + report configurations, and updated auth/proxy infrastructure for CSRF/session-based endpoints.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| src/app/modules/insights/default-reports/default-reports.component.ts | Main page logic: dependency wiring, chart setup, data loading, report config handling, export/download actions |
| src/app/modules/insights/default-reports/default-reports.component.html | Page UI: form controls, chart canvases, tables, export buttons, statistics accordion |
| src/app/modules/insights/default-reports/modal/report-config-modal.component.ts | Modal logic for create/rename/delete report configurations |
| src/app/modules/insights/default-reports/modal/report-config-modal.component.html | Modal UI for configuration actions |
| src/app/modules/insights/default-reports/modal/export-report-modal.component.ts | Modal logic for exporting report data to XLSX |
| src/app/modules/insights/default-reports/modal/export-report-modal.component.html | Export modal UI |
| src/app/modules/insights/default-reports/modal/index.ts | Barrel exports for modals |
| src/@seed/api/inventory-report/inventory-report.service.ts | Client wrapper for report, aggregated report, and report export endpoints |
| src/@seed/api/inventory-report/inventory-report.types.ts | Types for report chart points, aggregated points, stats payloads, etc. |
| src/@seed/api/inventory-report/index.ts | Barrel exports for inventory-report API |
| src/@seed/api/report-configuration/report-configuration.service.ts | CRUD + org-change auto-refresh for saved report configurations |
| src/@seed/api/report-configuration/report-configuration.types.ts | Types for report configurations and responses |
| src/@seed/api/report-configuration/index.ts | Barrel exports for report-configuration API |
| src/@seed/api/index.ts | Re-export new API modules |
| src/app/core/auth/auth.provider.ts | Configured Angular XSRF cookie/header names |
| src/app/core/auth/auth.interceptor.ts | Added CSRF cookie parsing/header injection for mutating API requests |
| proxy.conf.mjs | Rewrites Origin/Referer on proxied API requests |
| .spelling.dic | Added csrftoken to dictionary |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
… load conversion Agent-Logs-Url: https://github.com/SEED-platform/seed-angular/sessions/ea0c2cf7-5d75-4950-9ca3-129d5e35aadd Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com>
… error, remove redundant CSRF handling Agent-Logs-Url: https://github.com/SEED-platform/seed-angular/sessions/e265961d-9aad-44c3-be72-f06517de29ff Co-authored-by: kflemin <2205659+kflemin@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements the Default Reports page under Insights, porting the functionality from the
AngularJS inventory_reports_controller.js to Angular v21. This provides interactive
scatter and bar chart visualizations of property data across cycles and access levels.
Core Components:
bar), reactive form controls for axis selection, cycle/filter configuration, and a
statistics accordion
configurations
Services:
/report_export/ endpoints
auto-refresh on org change
Infrastructure:
X-CSRFToken header on mutating requests (required for Django views using
SessionAuthentication)